Skip to content

feat: added notify logic#2

Merged
cmainas merged 2 commits into
urunc-dev:mainfrom
7h3-3mp7y-m4n:notify
May 25, 2026
Merged

feat: added notify logic#2
cmainas merged 2 commits into
urunc-dev:mainfrom
7h3-3mp7y-m4n:notify

Conversation

@7h3-3mp7y-m4n
Copy link
Copy Markdown
Collaborator

Added the notify logic

The ANSI escape logic to get rid of those symbols, logs

Screenshot 2026-05-22 at 1 52 48 AM

Signed-off-by: 7h3-3mp7y-m4n <emailtorash@gmail.com>
Copy link
Copy Markdown
Contributor

@cmainas cmainas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @7h3-3mp7y-m4n ,

thank you for the fix, just one small comment.

Comment thread .github/workflows/updates.yaml Outdated
'. + {($id): ($runs[0] + {name: $name})}' \
runs_raw.json > tmp.json

mv tmp.json runs_raw.json
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's just fix the spacing here in the beginning of the line.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for sure!

Signed-off-by: 7h3-3mp7y-m4n <emailtorash@gmail.com>
Copy link
Copy Markdown
Contributor

@cmainas cmainas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @7h3-3mp7y-m4n .

@cmainas cmainas merged commit 53339a1 into urunc-dev:main May 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants